Skip to content

Update qcom-preflight-checks.yml#80

Open
Sandhya1236 wants to merge 1 commit intoqualcomm-linux:video.qclinux.0.0from
Sandhya1236:Upgrade-qc-preflight-checks
Open

Update qcom-preflight-checks.yml#80
Sandhya1236 wants to merge 1 commit intoqualcomm-linux:video.qclinux.0.0from
Sandhya1236:Upgrade-qc-preflight-checks

Conversation

@Sandhya1236
Copy link
Copy Markdown

This upgrades the QC Preflight Checks to the latest.

FYI there are 2 new optional checks that you may want to consider:

https://github.com/qualcomm/qcom-reusable-workflows?tab=readme-ov-file#additional-checks

@Sandhya1236 Sandhya1236 force-pushed the Upgrade-qc-preflight-checks branch from 4d5dd09 to beabf08 Compare March 10, 2026 07:57
@njjetha
Copy link
Copy Markdown

njjetha commented Mar 11, 2026

@Sandhya1236 I see that changes in file doesn't meet with the latest qcom-preflight-checks file

Copy link
Copy Markdown

@njjetha njjetha Mar 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

event need to be updated from pull_request_target to pull_request

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update the workflow name to QC Preflight Checks

uses: qualcomm/qcom-reusable-workflows/.github/workflows/qcom-preflight-checks-reusable-workflow.yml@v2
with:
# ✅ Preflight Checkers
repolinter: true # default: true
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

update all the parameters.

secrets:
SEMGREP_APP_TOKEN: ${{ secrets.SEMGREP_APP_TOKEN }} No newline at end of file

SEMGREP_APP_TOKEN: ${{ secrets.SEMGREP_APP_TOKEN }}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have removed SEMGREP_APP_TOKEN.

@github-advanced-security
Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@Sandhya1236 Sandhya1236 force-pushed the Upgrade-qc-preflight-checks branch from b3c5b25 to dc00243 Compare March 13, 2026 21:52
Upgrade to latest v2 QC Preflight Checks
Also adds optional checks defaulted to false

Signed-off-by: Sandhya Adavikolanu <sadaviko@qti.qualcomm.com>

Update qcom-preflight-checks.yml

Upgrade to latest QC Preflight Checks

Signed-off-by: Sandhya Adavikolanu <sadaviko@qti.qualcomm.com>
@Sandhya1236 Sandhya1236 force-pushed the Upgrade-qc-preflight-checks branch from dc00243 to e1c40d3 Compare March 13, 2026 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants